Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Filter Volumes by the State #1798

Closed
wants to merge 1 commit into from

Conversation

theputta
Copy link
Contributor

SUMMARY

Filter volumes based on their state, such as "in-use" or "available," using the status flag.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

amazon.aws.ec2_vol_info:
filters:
status:

Filter volumes based on their state, such as "in-use" or "available," using the status flag.
@github-actions
Copy link

Docs Build 📝

Thank you for contribution!✨

The docsite for this PR is available for download as an artifact from this run:
https://github.com/ansible-collections/amazon.aws/actions/runs/6497597081

You can compare to the docs for the main branch here:
https://ansible-collections.github.io/amazon.aws/branch/main

File changes:

  • M collections/amazon/aws/ec2_vol_info_module.html
Click to see the diff comparison.

NOTE: only file modifications are shown here. New and deleted files are excluded.
See the file list and check the published docs to see those files.

diff --git a/home/runner/work/amazon.aws/amazon.aws/docsbuild/base/collections/amazon/aws/ec2_vol_info_module.html b/home/runner/work/amazon.aws/amazon.aws/docsbuild/head/collections/amazon/aws/ec2_vol_info_module.html
index 512c1c8..03bfcdd 100644
--- a/home/runner/work/amazon.aws/amazon.aws/docsbuild/base/collections/amazon/aws/ec2_vol_info_module.html
+++ b/home/runner/work/amazon.aws/amazon.aws/docsbuild/head/collections/amazon/aws/ec2_vol_info_module.html
@@ -363,6 +363,11 @@ see <a class="reference internal" href="#ansible-collections-amazon-aws-ec2-vol-
 <span class="w">    </span><span class="nt">filters</span><span class="p">:</span>
 <span class="w">      </span><span class="nt">attachment.status</span><span class="p">:</span><span class="w"> </span><span class="l l-Scalar l-Scalar-Plain">attached</span>
 
+<span class="c1"># Gather information about volumes that are &quot;in-use&quot;</span>
+<span class="w">  </span><span class="nt">amazon.aws.ec2_vol_info</span><span class="p">:</span>
+<span class="w">    </span><span class="nt">filters</span><span class="p">:</span>
+<span class="w">      </span><span class="nt">status</span><span class="p">:</span><span class="w"> </span><span class="l l-Scalar l-Scalar-Plain">in-use</span>
+
 <span class="c1"># Gather information about all volumes related to an EC2 Instance</span>
 <span class="c1"># register information to `volumes` variable</span>
 <span class="c1"># Replaces functionality of `amazon.aws.ec2_vol` - `state: list`</span>

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/4e59f0babf7344c6b4ecdb77716bbcc8

✔️ ansible-galaxy-importer SUCCESS in 3m 53s
✔️ build-ansible-collection SUCCESS in 12m 43s
✔️ ansible-test-splitter SUCCESS in 4m 50s
✔️ integration-amazon.aws-1 SUCCESS in 14m 09s
Skipped 43 jobs

abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
black - aws_ssm

SUMMARY
The aws_ssm integration tests are currently broken (ansible-collections#1793), apply the black changes
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION
changes were approved as a part of ansible-collections#1784 but split because the tests were broken

Reviewed-by: Alina Buzachis
@tremble tremble added the docs label Nov 30, 2023
@GomathiselviS
Copy link
Contributor

I will close this PR because there has been no response to the comment. Could you consolidate all the suggested changes to the example section of the documentation and submit them in a single PR after validation?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants